[core][format] op format factory mv to cache .#4813
[core][format] op format factory mv to cache .#4813ranxianglei wants to merge 6 commits intoapache:masterfrom
Conversation
|
I tihink |
|
The significance of unified management is that many methods do not need to be implemented multiple times. Third-party file formats just rename |
|
That's not beckoning enough to make this change. If you think FormatFactoryUtils is too redundant, we can re-use more code. We can change FactoryUtil to change FormatFactoryUtil to : |
|
Can you fix this as what I mentioned? Still regard FileFormatFactory as an independent interface. |
That need to handle files in META-INFO/services also |
|
Sure, I am writing @leaves12138 |
|
done.Compatible with existing third-party factory implementations . |
We don't need to combine META-INF/services/org.apache.paimon.format.FileFormatFactory and META-INF/services/org.apache.paimon.factories.Factory. |
Purpose
from #4497
better impl for #4782
Linked issue: close #xxx
Tests
API and Format
Documentation